Skip to content

Comments

Fix release workflow and split CI test jobs for independent retries#682

Merged
laughingman7743 merged 4 commits intomasterfrom
fix/release-asset-gitignore
Feb 21, 2026
Merged

Fix release workflow and split CI test jobs for independent retries#682
laughingman7743 merged 4 commits intomasterfrom
fix/release-asset-gitignore

Conversation

@laughingman7743
Copy link
Member

@laughingman7743 laughingman7743 commented Feb 21, 2026

Summary

  • Fix release workflow uploading .gitignore as default.gitignore asset
    • The dist/* glob pattern matched the .gitignore file in the dist/ directory
    • Changed to explicit file patterns (dist/*.whl, dist/*.tar.gz) to only upload actual build artifacts
    • Cleaned up default.gitignore from all affected past releases (v3.19.0 through v3.29.1)
  • Split CI test jobs for independent retries
    • Split the single test job into 3 sequential jobs (testtest-sqlatest-sqla-async)
    • Each job sets a TEST_TYPE env var that tox-gh-actions uses to select the appropriate tox factor
    • Enables "Re-run failed jobs" in GitHub Actions UI to retry only the failed test suite
    • Sequential execution via needs dependencies preserves Athena query quota management

Test plan

  • Verify next release does not include default.gitignore in assets
  • Verify CI shows 3 separate job groups in GitHub Actions UI
  • Verify each job runs only its designated test suite
  • Verify "Re-run failed jobs" retries only the failed job

🤖 Generated with Claude Code

The `dist/*` glob pattern matched the `.gitignore` file in the dist/
directory, which GitHub API renamed to `default.gitignore` when
uploading as a release asset. Use explicit file patterns (*.whl and
*.tar.gz) to only upload actual build artifacts.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@laughingman7743 laughingman7743 marked this pull request as ready for review February 21, 2026 15:27
Split the single test job into 3 sequential jobs (test, test-sqla,
test-sqla-async) to enable "Re-run failed jobs" in GitHub Actions UI.
Each job sets a TEST_TYPE env var that tox-gh-actions uses to select
the appropriate tox factor.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@laughingman7743 laughingman7743 changed the title Fix release workflow uploading .gitignore as asset Fix release workflow and split CI test jobs for independent retries Feb 21, 2026
laughingman7743 and others added 2 commits February 22, 2026 01:11
- Extract common test steps into test-suite.yaml reusable workflow
- Change test trigger from push to pull_request to avoid unnecessary
  CI runs on branch pushes and tag pushes
- Weekly schedule trigger retained for periodic master validation

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Move permissions (id-token: write) to the caller workflow so
the reusable workflow inherits the required OIDC token permission
for AWS credential configuration.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@laughingman7743 laughingman7743 merged commit 324e66f into master Feb 21, 2026
15 checks passed
@laughingman7743 laughingman7743 deleted the fix/release-asset-gitignore branch February 21, 2026 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant